Skip to content

Conversation

@dimodi
Copy link
Contributor

@dimodi dimodi commented Jul 16, 2025

No description provided.

@dimodi dimodi added the merge-to-production Use this label to get a comment to choose whether to merge the PR to production label Jul 16, 2025
@dimodi dimodi requested review from a team as code owners July 16, 2025 12:33
@github-actions
Copy link
Contributor

Hello @dimodi,

Check the below option if you would like to automatically generate PR to production. The automation uses the branch for the cherry-pick, and then will delete the branch. Please, do not delete it manually.

  • create PR to production

@dimodi dimodi merged commit 3f4313f into master Jul 16, 2025
1 check passed
@github-actions
Copy link
Contributor

Here you are the link to your PR to production: #3114

@github-actions github-actions bot deleted the grid-editing branch July 16, 2025 12:51
Copy link
Contributor

@ikoevska ikoevska left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed.

* Direct casting of the `context` can make two-way data binding not work properly.

<div class="skip-repl"></div>
>caption Not recommended: direct casting with two-way parameter binding
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typically, documentation avoids documenting "not recommended" scenarios. What's the reason to cover it here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-to-production Use this label to get a comment to choose whether to merge the PR to production

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants